-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fix broken link in Cloudflare docs #14017
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Fix broken link in Cloudflare docs #14017
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Located the source of the Cloudflare
Here is the breadcrumbs boolean value that was correctly documented by cursor It appears cursor properly read this file, or had some kind of RAG knowledge of it. The |
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a couple suggestions, looks good!
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
Bundle ReportChanges will increase total bundle size by 270 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
[Cursor created PR - Reviewed by me] Broken links in the Cloudflare configuration documentation have been resolved. * A new file, `docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx`, was created. * This addresses a 404 error caused by a missing page referenced by `fetchIntegration` in the integrations table. * The file includes metadata, documentation for `Sentry.fetchIntegration`, configuration options (`breadcrumbs`, `shouldCreateSpanForRequest`), and usage examples, specifically noting its relevance to Cloudflare Workers. * The link for `requestDataIntegration` in `platform-includes/configuration/integrations/javascript.cloudflare.mdx` was updated. * The previous link, `./requestDataIntegration`, had incorrect casing and pointed to a non-existent path. * The link was corrected to `./requestdata` to match the actual filename `requestdata.mdx`, ensuring consistency with other integration links. All links in the Cloudflare integrations table now correctly point to existing documentation files. --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
[Cursor created PR - Reviewed by me]
Broken links in the Cloudflare configuration documentation have been resolved.
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
, was created.fetchIntegration
in the integrations table.Sentry.fetchIntegration
, configuration options (breadcrumbs
,shouldCreateSpanForRequest
), and usage examples, specifically noting its relevance to Cloudflare Workers.requestDataIntegration
inplatform-includes/configuration/integrations/javascript.cloudflare.mdx
was updated../requestDataIntegration
, had incorrect casing and pointed to a non-existent path../requestdata
to match the actual filenamerequestdata.mdx
, ensuring consistency with other integration links.All links in the Cloudflare integrations table now correctly point to existing documentation files.